home *** CD-ROM | disk | FTP | other *** search
/ NetNews Offline 2 / NetNews Offline Volume 2.iso / news / comp / lang / c-part1 / 8723 < prev    next >
Encoding:
Text File  |  1996-08-05  |  1.9 KB  |  74 lines
  1. Path: news.lpr.carel.fi!usenet
  2. From: Ari Lukumies <aril@cmt.lpr.mail.carel.fi>
  3. Newsgroups: comp.lang.c
  4. Subject: Re: revised code of calling a function twice from printf
  5. Date: Tue, 05 Mar 1996 13:36:36 +0200
  6. Organization: Carelcomp Forest
  7. Message-ID: <313C2744.7FE7@cmt.lpr.mail.carel.fi>
  8. References: <4hfs54$k4e@newsbf02.news.aol.com>
  9. NNTP-Posting-Host: renoir.cclahti.carel.fi
  10. Mime-Version: 1.0
  11. Content-Type: text/plain; charset=us-ascii
  12. Content-Transfer-Encoding: 7bit
  13. X-Mailer: Mozilla 2.0 (WinNT; I)
  14.  
  15. Razine wrote:
  16. > 
  17. > Here is the actual code , modified with everyone's suggestions.  However
  18. > there is still a bug in here somewhere.  Can anyone please explain what is
  19. > wrong with this.
  20. > 
  21. > It returns
  22. > 
  23. > [1] NONE       [2] NE
  24. > 
  25. > I would like it to return
  26. > 
  27. > [1] NONE       [2] Asprin
  28. > 
  29. > Thank you very much for your help.  it is greatly appreciated.
  30. > 
  31. > #include <stdio.h>
  32. > #include <string.h>
  33. > 
  34. > char *display_drug_type(int drug_index);
  35. > 
  36. > int drug_inventory[5]={0,1,0,0,0};
  37. > 
  38. > int main() {
  39. > 
  40. >     printf("[1] %s             [2] %s
  41. > \n",display_drug_type(0),display_drug_type(1));
  42. > 
  43. > return 0;
  44. >         }
  45. > 
  46. > char *display_drug_type(int drug_index) {
  47. >    char drug_type[81]="\0";
  48. > 
  49. >    switch(drug_inventory[drug_index]) {
  50. >      case 0 : strcpy(drug_type,"NONE"); break;
  51. >      case 1 : strcpy(drug_type,"Asprin"); break;
  52. >      default : printf("Error in Display_drug_inventory");
  53. >                                                 }
  54. >    return drug_type;
  55. > }
  56.  
  57. In the printf, you call the same function twice. The order in which it gets called is 
  58. undetermined, thus the contents of drug_type will be undetermined. Why not rewrite 
  59. display_drug_type as:
  60.  
  61.     char    *display_drug_type(int drug_index)
  62.     {
  63.         switch (drug_inventory[drug_index]) {
  64.             case 0: return "NONE";
  65.             case 1: return "Asprin";
  66.         }
  67.         return "Invalid";
  68.     }
  69.  
  70. Later,
  71.  AriL
  72. -- 
  73. All my opinions are mine and mine alone.
  74.